Skip to content

Fix CPAN random module compatibility gaps#862

Merged
fglock merged 1 commit into
masterfrom
fix/cpan-random-modules-20260618
Jun 18, 2026
Merged

Fix CPAN random module compatibility gaps#862
fglock merged 1 commit into
masterfrom
fix/cpan-random-modules-20260618

Conversation

@fglock

@fglock fglock commented Jun 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • Preserve numified scalar state so Data::Dumper::Compact dumps numified ten-digit strings like Perl.
  • Replace the bundled Data::Dump stub with the full implementation needed by Data::Dump::Partial.
  • Fix SVG::Parser filehandle detection by passing typeglobs from XML::Parser::parsefile.
  • Return Data::UUID binary UUIDs as byte strings so Data::GUID and CPAN::Testers::Report pass.
  • Keep compiled qr// patterns isolated from outer s/// modifiers, fixing DateTimeX::Easy via DateTime::Format::Flexible.

Test Plan

  • make
  • prove src/test/resources/unit/regex/qr_substitution_flags.t with system Perl
  • ./jperl src/test/resources/unit/regex/qr_substitution_flags.t
  • ./jcpan -t Data::Dumper::Compact
  • ./jcpan -t Data::Dump::Partial
  • ./jcpan -t SVG::Parser
  • ./jcpan -t CPAN::Testers::Report
  • ./jcpan -t DateTimeX::Easy

Generated with Codex

Fix several CPAN random tester regressions:
- preserve numified scalar state for Data::Dumper output
- bundle the full Data::Dump implementation needed by Data::Dump::Partial
- pass typeglobs from XML::Parser::parsefile for SVG::Parser compatibility
- return Data::UUID binary UUIDs as byte strings
- keep compiled qr// flags isolated when used in s/// with outer modifiers

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock fglock merged commit 34866d5 into master Jun 18, 2026
2 checks passed
@fglock fglock deleted the fix/cpan-random-modules-20260618 branch June 18, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant